Skip to content

docs + chore: CLAUDE.md v0.35.3 progress + gitignore internal docs#413

Open
Nathan Schram (nathanschram) wants to merge 4 commits into
masterfrom
docs/claude-md-v0.35.3-progress
Open

docs + chore: CLAUDE.md v0.35.3 progress + gitignore internal docs#413
Nathan Schram (nathanschram) wants to merge 4 commits into
masterfrom
docs/claude-md-v0.35.3-progress

Conversation

@nathanschram
Copy link
Copy Markdown
Member

@nathanschram Nathan Schram (nathanschram) commented Apr 22, 2026

Summary

Two-in-one small-PR combining docs note + public-repo hygiene gitignore pass.

Commit 1 — f440f91 docs(claude.md): note v0.35.3rc1 staging + Claude extra_args feature

Commit 2 — 90a2df6 chore(gitignore): untrack internal docs

Public repo hygiene. Three classes of file now gitignored going forward:

  1. docs/handover/ (new) — Claude Code handover docs between sessions. Internal-only.
  2. docs/audits/ — incident and security audits referencing production bot names + internal workflows. GitHub issues/milestones are the public tracker.
  3. docs/tests/ — per-release integration test plans and execution reports. Contain internal bot refs, skip-notes, QA methodology. docs/reference/integration-testing.md remains the public playbook.
  4. incoming/*.md — draft design/feedback markdown uploaded via Telegram file-transfer. incoming/file_*.jpg auto-uploads also covered.

Files untracked from index (contents preserved on disk):

  • docs/audits/pitchdocs-context-guard-interference.md
  • docs/tests/v0.35.2-integration-test-plan.md
  • docs/tests/results/v0.35.2-results.md
  • docs/tests/results/v0.35.2rc3-results.md

Docs site impact

None. zensical.toml nav does not reference audits/, tests/, or handover/. Build unaffected.

History caveat

These files remain visible in git history. Full scrub would require BFG/filter-branch — out of scope for this PR. Forward-going commitment is that new internal planning/audit/test-artifact content stops being tracked.

Test plan

  • git check-ignore -v confirms all four new patterns match expected paths
  • docs/audits/pitchdocs-*.md, docs/tests/*, docs/handover/*, incoming/*.md all now gitignored
  • Previously tracked files removed from index with git rm --cached (contents preserved)
  • zensical.toml nav audit — no references to removed folders
  • CI green on this PR

🤖 Generated with Claude Code

Enables `[claude] extra_args = ["--chrome"]` so Untether-spawned Claude
Code sessions can opt into the Claude-in-Chrome extension — previously
the `mcp__claude-in-chrome__*` tool namespace was absent from Untether
sessions because Claude Code 2.1.x gates it behind `--chrome` /
`CLAUDE_CODE_ENABLE_CFC=1`, and Untether never passed the flag.

Mirrors `codex.extra_args` and `pi.extra_args`. Flags Untether manages
internally (`-p`, `--print`, `--output-format`, `--input-format`,
`--resume`/`-r`, `--continue`/`-c`, `--permission-mode`,
`--permission-prompt-tool`) are rejected at config-load with a
`ConfigError` so duplicate-argv surprises fail fast. User args land on
argv after the managed stream-json prelude and before resume / model /
effort / allowed-tools / permission flags, preserving the trailing
`-p <prompt>` (or stdin prompt under permission-mode) position.

- src/untether/runners/claude.py: add `extra_args` field, thread
  through `build_args`, parse + validate in `build_runner`
- tests/test_build_args.py: +8 tests (argv ordering, permission-mode
  argv, multi-flag order, build_runner parsing, reserved-flag rejection
  for individual flags and `key=value` prefixes)
- docs/reference/config.md, docs/reference/runners/claude/runner.md:
  document the new key, including reserved-flag list
- CHANGELOG.md: v0.35.3 (unreleased) entry

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* chore: staging 0.35.3rc1

Stage Claude extra_args (#407) for TestPyPI. This rc1 is the wheel the Mac
Untether instance will install to validate Claude-in-Chrome end-to-end per
docs/audits/2026-04-21-claude-in-chrome-test-plan.md.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* deps: bump lxml 6.0.2→6.1.0 and python-dotenv 1.2.1→1.2.2

pip-audit flagged two new transitive CVEs after PR #408 merged:
- lxml 6.0.2: CVE-2026-41066 (fix 6.1.0) — pulled via sulguk
- python-dotenv 1.2.1: CVE-2026-28684 (fix 1.2.2) — pulled via
  pydantic-settings

Both have clean fixes. Lockfile-only change; pyproject.toml constraints
unchanged. Local pip-audit clean after bump.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- Bump unit-test count 2372 → 2387 (reflects #407 +8 test_build_args tests
  and prior untracked test additions).
- Expand test_build_args.py entry 42 → 56 tests with the new coverage areas.
- Add extra_args passthrough feature entry under "Features (vs upstream
  takopi)" — documents the Claude-in-Chrome motivator, reserved-flag list,
  and argv placement (#407, shipped in v0.35.3rc1).

Issue progress tracked in gh#407 comment.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 22, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 22267f1e-d43e-47dc-afbf-f16488611925

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/claude-md-v0.35.3-progress

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

…dovers

Public repo hygiene pass. Three classes of file shouldn't be committed
going forward:

1. docs/handover/ (new) — Claude Code handover docs that pass context
   between sessions. Internal-only by nature.
2. docs/audits/ — incident and security audits referencing production
   bot names and internal workflows. GitHub issues/milestones are the
   public tracker.
3. docs/tests/ — per-release integration test plans and execution
   reports. Contain internal bot references, skipped-test notes, and
   QA methodology detail that isn't user-facing. docs/reference/
   integration-testing.md remains the public playbook.
4. incoming/*.md — draft design and feedback markdown uploaded via
   Telegram file-transfer. Auto-named file_*.jpg already covered.

Files untracked (contents preserved on disk):
- docs/audits/pitchdocs-context-guard-interference.md
- docs/tests/v0.35.2-integration-test-plan.md
- docs/tests/results/v0.35.2-results.md
- docs/tests/results/v0.35.2rc3-results.md

mkdocs/zensical nav (zensical.toml) doesn't reference any of these
paths, so the docs site build is unaffected.

History note: these files remain visible in git history; removing
them entirely would require a separate BFG/filter-branch pass, which
is out of scope for this PR. The forward-going commitment is that
internal planning/audit/test-artifact content stops being tracked.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@nathanschram Nathan Schram (nathanschram) changed the title docs(claude.md): note v0.35.3rc1 staging + Claude extra_args feature docs + chore: CLAUDE.md v0.35.3 progress + gitignore internal docs Apr 22, 2026
Base automatically changed from dev to master May 26, 2026 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant